Skip to content

Fix two-frame script delay when respawning#1258

Open
NyakoFox wants to merge 1 commit intoTerryCavanagh:masterfrom
NyakoFox:death-delay-fix
Open

Fix two-frame script delay when respawning#1258
NyakoFox wants to merge 1 commit intoTerryCavanagh:masterfrom
NyakoFox:death-delay-fix

Conversation

@NyakoFox
Copy link
Contributor

@NyakoFox NyakoFox commented Dec 3, 2025

Changes:

At some point, there was some "kludge" added to the game which allowed script boxes to immediately get triggered when you enter a room, rather than the update order unfortunately causing scripts to trigger after two frames.

For some reason, dying and respawning in a room does not have the same fix, causing really bad inconsistencies for anything that needs precise timing.

Despite the initial implementation of the kludge being somewhat careless and breaking levels which relied on timing, this one should not break anything, but in fact fix things, surprisingly.

Legal Stuff:

By submitting this pull request, I confirm that...

  • My changes may be used in a future commercial release of VVVVVV
  • I will be credited in a CONTRIBUTORS file and the "GitHub Friends"
    section of the credits for all of said releases, but will NOT be compensated
    for these changes unless there is a prior written agreement

At some point, there was some "kludge" added to the game which allowed
script boxes to immediately get triggered when you enter a room, rather
than the update order unfortunately causing scripts to trigger after two
frames.

For some reason, dying and respawning in a room does not have the same
fix, causing really bad inconsistencies for anything that needs precise
timing.

Despite the initial implementation of the kludge being somewhat careless
and breaking levels which relied on timing, this one should not break
anything, but in fact fix things, surprisingly.
Copy link
Contributor

@Daaaav Daaaav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes sense and seems to work nicely!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants